Conversation
benelan
requested review from
a team,
driskull,
geospatialem and
jcfranco
as code owners
github-actions
Bot
added
the
chore
benelan
commented
Jul 14, 2023
|
|
||
| .text { | ||
| @apply mt-px mb-0; | ||
| margin-inline-start: theme("spacing.px"); |
Contributor
Author
There was a problem hiding this comment.
@eriklharper can you confirm that eslint or prettier removed the correct one of these duplicate css properties?
Contributor
Author
There was a problem hiding this comment.
I blamed it and it's from this commit: 4169f30
The one not being used was removed so I assume that's correct, but just wanted to double check before its gone forever.
benelan
added a commit
that referenced
this pull request
Jul 17, 2023
…script (#7326) **Related Issue:** #7319 ## Summary Disables the [`jsdoc/require-param-type`](https://github.com/gajus/eslint-plugin-jsdoc/blob/main/docs/rules/require-param-type.md) eslint rule since we use TypeScript for that.
benelan
added a commit
that referenced
this pull request
Jul 18, 2023
**Related Issue:** #7318 ## Summary Lint and format the code. Prettier will now add trailing commas when possible, which is the default in 3x. I know already brought up disabling some jsdoc rules in the past, but I think we can at least disable the ` jsdoc/require-param-type` one. We are using TypeScript so defining the types twice seems redundant and likely to get out of sync when things change.
benelan
added a commit
that referenced
this pull request
Jul 18, 2023
…amping-ar-numberingsystem-bug * origin/main: chore(eslint): disable jsdoc/require-param-type rule in favor of typescript (#7326) style: lint and format code (#7319) chore: add editorconfig (#7318) chore: release next fix(input-date-picker): provides placeholder text context for AT users (#7320) chore: release next fix(dropdown-item): provides accessible label when href is not parsed (#7316) ci: create more specific codeowners to limit unnecessary notifications (#7313)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related Issue: #7318
Summary
Lint and format the code. Prettier will now add trailing commas when possible, which is the default in 3x.
I know already brought up disabling some jsdoc rules in the past, but I think we can at least disable the
jsdoc/require-param-typeone. We are using TypeScript so defining the types twice seems redundant and likely to get out of sync when things change.